-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add pipefail to space_robots build so that it fails on error #76
Conversation
6f2b402
to
501481f
Compare
This fails the space_robots build as expected! |
Also updated |
Rebasing to |
b80e74f
to
fca11ab
Compare
@EzraBrooks - can you review and approve? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. always one of the first low-hanging fruits I attack on shell scripts too :)
Following a minimal set of rules to track changes, associate them to milestones and releases, and determine the process of accepting them, will help us comply with processes existing at NASA (e.g., NPR7150.2D) and other organizations for Space ROS to be accepted in space software applications. This commit documents in the CONTRIBUTING file the process agreed upon by the Space ROS Technical Committee.
To increase visibility of the contributing guidelines and facilitate early adoption by potential contributors, this commit adds a link to the CONTRIBUTING.md file from the top-level README.
…ules Document contribution rules (space-ros#76).
Following a minimal set of rules to track changes, associate them to milestones and releases, and determine the process of accepting them, will help us comply with processes existing at NASA (e.g., NPR7150.2D) and other organizations for Space ROS to be accepted in space software applications. This commit documents in the CONTRIBUTING file the process agreed upon by the Space ROS Technical Committee.
To increase visibility of the contributing guidelines and facilitate early adoption by potential contributors, this commit adds a link to the CONTRIBUTING.md file from the top-level README.
…ntrib-rules Document contribution rules (space-ros/docker#76).
Add pipefail to space_robots build so that it fails on error
Add pipefail to space_robots build so that it fails on error
Adding
set -eo pipefail
to the space_robots workflow to fix #75